-
Notifications
You must be signed in to change notification settings - Fork 695
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add nf-test and fix stub happy/sompy #6285
add nf-test and fix stub happy/sompy #6285
Conversation
{ assert snapshot(process.out.stats).match("stats_1") }, | ||
{ assert snapshot(process.out.versions).match("versions_1") }, | ||
{ assert snapshot(file(process.out.metrics[0][1]).name).match("metrics_1")}, | ||
{ assert snapshot(file(process.out.features[0][1]).name).match("features_1")} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you put all the snapshots into one: (indentation probably needs fixing)
{ assert snapshot(process.out.stats).match("stats_1") }, | |
{ assert snapshot(process.out.versions).match("versions_1") }, | |
{ assert snapshot(file(process.out.metrics[0][1]).name).match("metrics_1")}, | |
{ assert snapshot(file(process.out.features[0][1]).name).match("features_1")} | |
{ assert snapshot( | |
process.out.stats, | |
process.out.versions, | |
file(process.out.metrics[0][1]).name, | |
file(process.out.features[0][1]).name | |
).match() }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when I don't separate "versions", lint gives an error. that is why I put it like this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, shouldn't be an issue like this, I've done it on a number of modules
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont know what was that, but it works out inside one match now
PR checklist
Closes #XXX
versions.yml
file.label
nf-core modules test <MODULE> --profile docker
nf-core modules test <MODULE> --profile singularity
nf-core modules test <MODULE> --profile conda
nf-core subworkflows test <SUBWORKFLOW> --profile docker
nf-core subworkflows test <SUBWORKFLOW> --profile singularity
nf-core subworkflows test <SUBWORKFLOW> --profile conda